[pull] master from broxus:master#16
Merged
pull[bot] merged 80 commits intovenom-blockchain:masterfrom Mar 19, 2025
Merged
Conversation
Add a new contract wallet multisig 2.1
Add support for pruned cells to allow signing a large transactions by Ledger
Implement 'Confirm' for multisig wallets; fix parsing token payload
Add ability to sign transaction with chain id
- Use clang-format to format all .c and .h files
- remove AlignConsecutiveAssignments: true - remove AlignConsecutiveDeclarations: true
…-build-compilation-on-nano-d Fix/ldg 682 nano app get a clean build and compilation on nano
…implementations This commit introduces a new UI display architecture: - Created a common display.h header with UI display function declarations - Added platform-specific implementations for BAGL and NBGL - Moved UI flow definitions from individual handler files to display_bagl.c - Extracted response setting functions to a separate response_setter.c/h - Updated handler files to use new centralized UI display functions - Replaced direct ux_flow_init calls with platform-agnostic UI display functions
…nfirmation instructions
…get-address-on-stax-flex Feat/ Implement NBGL display
…te golden snapshots
…and make changes accordingly [LDG-681] - Create 'approve' and 'reject' screen and use them on relevant flows - Switch the position of 'reject step' and 'approve step' for all flows - Edit the sign_tx tests to look for 'Approve' instead of 'Accept' - Generate golden snapshots
…tial-screen-for-get_public_k Fix/ldg 680 nano app missing initial screen for get public k
This commit removes the entire `old-tests` directory, which contained a legacy test suite for the Ledger app. The directory included Rust and Python test files, Speculos screenshot tests, and various configuration and client-side testing utilities.
Update binary images for Everscale app icons in 32px and 40px sizes, and glyph in 64px size
Regenerate golden snapshots for Flex device UI tests, including scenarios for address wallet, public key, message signing, and transaction signing flows
Update golden snapshots for Stax device UI tests, covering various scenarios including address wallet, public key, message signing, and transaction signing flows
…nano-app-32px-40px-64px- Feat/ldg 685 generate icons for nano app 32px 40px 64px
… building instructions - Updated Makefile to include configuration for Venom variant, setting application name and icons accordingly. - Enhanced README with detailed instructions for building both Everscale and Venom variants of the application.
- Incremented application version from 1.1.0 to 1.1.1. - Modified Makefile to conditionally set VARIANT_VALUES based on COIN variable, ensuring proper configuration for Venom and Ever variants. - Added new glyph and icon files for Venom variant in various sizes.
…-build-to-the-app Feat/LDG-693--nano-app-add-venom-build-to-the-app
Rename app_venom_32px.gif.gif to app_venom_32px.gif
Comment on lines
+109
to
+113
| // P2_MORE is a signal for more apdu to receive in current chunk; | ||
| // P2_EXTEND is a signal for extended buffer and can't be in first chunk; | ||
| // P2_MORE && !P2_EXTEND = first chunk; | ||
| // P2_EXTEND && !P2_MORE = last chunk; | ||
| // P2_EXTEND && P2_MORE = ordinary request without chunks; |
Check notice
Code scanning / CodeQL
Commented-out code Note
Copilot Autofix
AI 12 months ago
Copilot could not generate an autofix suggestion
Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )